Skip to content

Use setup_cloud_environment() for OpenStack connections in manage com…#1900

Merged
berendt merged 1 commit intomainfrom
fix-os-cloud
Dec 5, 2025
Merged

Use setup_cloud_environment() for OpenStack connections in manage com…#1900
berendt merged 1 commit intomainfrom
fix-os-cloud

Conversation

@berendt
Copy link
Copy Markdown
Member

@berendt berendt commented Dec 5, 2025

…mands

Replace direct get_cloud_connection() calls with setup_cloud_environment() and cleanup_cloud_environment() pattern in all manage commands. This ensures proper cloud configuration by:

  • Copying /etc/openstack/clouds.yaml to /tmp
  • Loading passwords securely from vault via get_cloud_password()
  • Creating /tmp/secure.yml with decrypted credentials
  • Properly cleaning up temporary files after execution

Affected commands:

  • manage loadbalancer list/reset/delete
  • manage amphora restore/rotate
  • manage volume list/repair
  • manage server migrate/list/clean

Also adds --cloud parameter to ServerMigrate for consistency.

AI-assisted: Claude Code

…mands

Replace direct get_cloud_connection() calls with setup_cloud_environment()
and cleanup_cloud_environment() pattern in all manage commands. This ensures
proper cloud configuration by:
- Copying /etc/openstack/clouds.yaml to /tmp
- Loading passwords securely from vault via get_cloud_password()
- Creating /tmp/secure.yml with decrypted credentials
- Properly cleaning up temporary files after execution

Affected commands:
- manage loadbalancer list/reset/delete
- manage amphora restore/rotate
- manage volume list/repair
- manage server migrate/list/clean

Also adds --cloud parameter to ServerMigrate for consistency.

AI-assisted: Claude Code

Signed-off-by: Christian Berendt <berendt@osism.tech>
@berendt berendt merged commit 02c8af8 into main Dec 5, 2025
2 checks passed
@berendt berendt deleted the fix-os-cloud branch December 5, 2025 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant